chore(contributor-check): bump AGT pin to 359a6b8 - #26
Merged
Conversation
The pin was 410fb2cc (2026-06-09), so every agentrust-io repo has been
running detection logic roughly two months stale.
The change that matters is the fail-closed risk ordering. At the old pin
RISK_ORDER was {LOW:1, MEDIUM:2, HIGH:3, UNKNOWN:0}, so UNKNOWN sorted
BELOW LOW and never met the MEDIUM comment threshold. A check that
errored or got rate-limited was therefore silently dropped and read as
clean. It is now {LOW:1, MEDIUM:2, UNKNOWN:3, HIGH:4}: an undetermined
result outranks LOW and MEDIUM and gets surfaced.
Also picks up ~360 lines of contributor_check.py hardening: org-backed
and prior-interaction credibility tiers, the maintainer allowlist that
can only soften HIGH to MEDIUM and never grant LOW, and guards against
star-bought repos and two-account merge rings.
Verified no breaking change: contributor_check_action.py CLI arguments
are unchanged, and .github/actions/contributor-check/ upstream is
byte-identical between the two pins. brand_comment.py matches the footer
by prefix, so branding still applies.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The AGT pin was
410fb2cc(2026-06-09). Every agentrust-io repo has beenrunning contributor detection logic about two months stale.
The bug this fixes
At the old pin:
UNKNOWNsorted belowLOW, so it never met the MEDIUM commentthreshold. A check that errored, or got rate-limited by the GitHub API, was
silently dropped and effectively read as a clean result. Current:
An undetermined result now outranks LOW and MEDIUM and gets surfaced for a
human to look at. Fail-closed (upstream microsoft/agent-governance-toolkit#2950).
Also picked up
Roughly 360 lines of
contributor_check.pyhardening: org-backed andprior-interaction credibility tiers, the maintainer allowlist that can only
soften a HIGH auto-flag to MEDIUM and can never grant LOW, and guards against
star-bought repos and two-account merge rings.
Verified no breaking change
contributor_check_action.pyCLI arguments are unchanged between the pins..github/actions/contributor-check/upstream is byte-identical between thetwo pins.
brand_comment.pymatches the footer by line prefix, not exact text, soAgenTrust branding still applies.
Propagation, and a separate problem
7 of the 8 consuming repos check this repo out with
ref: main, so they pickthis up as soon as it merges.
awesome-ai-governanceis pinned to0b440ffand will not.
That mutable
ref: mainis itself worth fixing. These arepull_request_targetworkflows holdingissues: writeandpull-requests: write, and this action's own template comment says the ref"must not be a mutable ref". Raising separately rather than bundling it, since
pinning 8 repos to SHAs is a maintenance-model decision, not a one-line change.